Testament hybrid runner#1724
Open
bung87 wants to merge 17 commits into
Open
Conversation
…ared-state tests
- Migrate from monolithic tester.nim to hybrid approach:
- testament runs isolatable tests individually for faster feedback
- tester.nim preserves execution order and shared nimbleDir state
for tests that require network or mutate global state
- Split 13 local-only tests from tissues.nim into testament files:
tests/tissues/tissue_{108,304,349,351,564,581,597,633,793,941,1609,1636,test_params}.nim
- Remove already-split tests from tester.nim imports and skip.txt:
tcheckcommand, tinitcommand, tcleancommand, tnimbletasks, ttestcommand,
tforgeparser, tnimbledump, tnimscript, tpathcommand, toffline, tlocaldeps
- Update nimble.nimble test task to run both testament suites
…tdeps to testament format
…rser; isolate tnimbinaries
…n nim pkg installed in nimbleDir)
- tissue_1648: task deps should not install newer dependencies - tissue_1650: root project config.nims should not leak into dep binary build - tissue_428: local nimbleDir install with -g - tconfignewline: config file should end with newline - tnonimblefile: friendly error when running command without .nimble file - trecursivetasks: tasks can be called recursively - tnimbleversion: NimbleVersion defined in compiled package - truncommand_local: 12 basic nimble run parameter-parsing tests Also fixes tshellenv.nim: revert accidental testutils -> common rename introduced when renaming testscommon -> common.
testament --directory:tests runs from tests/, so relative config.nims was not found. Use currentSourcePath().parentDir.parentDir instead.
Removed tests that used sequtils, causing 'unused import' warning that fails the 'compilation without warnings' test.
- Extract 19 pure in-memory SAT solver tests from tsat.nim into tests/sat/tsat_solver.nim (zero network, zero shared state) - tsat.nim now only contains 3 tests that download/install packages: - nitter fork URL resolution - issue nim-lang#1162 integration (conflictingdepres) - libp2pconflict fallback - Add testament --directory:tests/sat to nimble.nimble test task - Trim unused imports in both files
- tasynctools.nim never imports common.nim and operates entirely in getTempDir() — zero shared state, zero nimbleDir pollution - Add discard exitcode header for testament standalone mode - Remove from tester.nim imports and skip.txt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
just show the progress before going further.